Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Java_toolchain: Remove Java 7 compatibility mode #9450

Conversation

davido
Copy link
Contributor

@davido davido commented Sep 29, 2019

Closes #8772.

Change-Id: Ia3467b2a33ee4bfbd10cfe3f1027e292d3a967b7

Closes bazelbuild#8772.

Change-Id: Ia3467b2a33ee4bfbd10cfe3f1027e292d3a967b7
@davido
Copy link
Contributor Author

davido commented Oct 3, 2019

Superseded with: #9494

@davido davido closed this Oct 3, 2019
bazel-io pushed a commit that referenced this pull request Oct 7, 2019
Closes #8772.

Change-Id: Ia3467b2a33ee4bfbd10cfe3f1027e292d3a967b7

Closes #9450.

Change-Id: Ia3467b2a33ee4bfbd10cfe3f1027e292d3a967b7
PiperOrigin-RevId: 273233710
lucamilanesio pushed a commit to GerritCodeReview/gerrit that referenced this pull request Oct 13, 2019
This upgrade of java_tools is needed to remove Java 7 compatibility mode
for java_proto_library rules. This was done in this CL: [1], that is
included in java_tools release 6.0.

[1] bazelbuild/bazel#9450

Bug: Issue 11102
Change-Id: Ic0043544d238ee5cc2dcaf67ff1db9d8a5397bfa
thaidn added a commit to tink-crypto/tink that referenced this pull request Jul 14, 2020
The primary reason to drop that support is because Bazel [1] and Protobuf [2] have stopped supporting Java 7.

The second reason is that nobody in the wild should rely on Java 7 any more because it was discontinued years ago.

The third reason is, due to the first reason, Tink 1.4.0-rc1 and 1.4.0-rc2 were accidentally compiled with Java 8. No user, however, has complained.

Remove the JAVACOPTS option. Because Tink depends on Protobuf, Tink cannot really specify which Java version it can support, but it has to use whatever version that Protobuf supports.

[1] bazelbuild/bazel#9450
[2] protocolbuffers/protobuf#7306

PiperOrigin-RevId: 321114818
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cla: yes team-Rules-Java Issues for Java rules
Projects
None yet
Development

Successfully merging this pull request may close these issues.

java_proto_library results in spurious warning message
3 participants